Skip to content

Help background fill autest to run on ubuntu #10936

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
duke8253 merged 1 commit into from
Apr 29, 2024
Merged

Help background fill autest to run on ubuntu #10936

duke8253 merged 1 commit into from
Apr 29, 2024

Conversation

@duke8253
Copy link
Contributor

@duke8253 duke8253 commented Dec 14, 2023

this resolves #10862

@duke8253 duke8253 added AuTest Ubuntu Bugs related to Ubuntu Linux labels Dec 14, 2023
@duke8253 duke8253 added this to the 10.0.0 milestone Dec 14, 2023
@duke8253 duke8253 requested a review from shinrich December 14, 2023 17:18
@duke8253 duke8253 self-assigned this Dec 14, 2023
@duke8253 duke8253 changed the title Help autest to run on ubuntu Help background fill autest to run on ubuntu Dec 14, 2023
@duke8253 duke8253 force-pushed the master-ubuntu_bg_fill branch from 6ff08e8 to dac5ca1 Compare December 15, 2023 19:10
@bryancall bryancall requested a review from moonchen January 8, 2024 23:55
@duke8253 duke8253 force-pushed the master-ubuntu_bg_fill branch from dac5ca1 to da9d40a Compare January 29, 2024 16:22
@moonchen
Copy link
Contributor

moonchen commented Jan 30, 2024

I'm getting some failures in on Ubuntu 23.04 running in UTM.

results2.txt

ATS is built as follows:

CC=clang-17 CXX=clang++-17 cmake -GNinja -DCMAKE_INSTALL_PREFIX:PATH=/opt/ats2 \
    -DCMAKE_BUILD_TYPE=Debug \
    -DENABLE_CCACHE=ON \
    -DENABLE_ASAN=OFF \
    -DCMAKE_EXPORT_COMPILE_COMMANDS=ON \
    -DENABLE_AUTEST=ON \
    ..

cmake --build .

cmake --install .

Calling autest:

./autest.sh --filter=background_fill --ats-bin=/opt/ats2/bin | sed -r "s/\x1B\[([0-9]{1,3}(;[0-9]{1,2};?)?)?[mGK]//g" > /tmp/results2 2>&1

@duke8253 duke8253 force-pushed the master-ubuntu_bg_fill branch from da9d40a to 06c0197 Compare January 31, 2024 18:54
@shinrich
Copy link
Member

shinrich commented Feb 9, 2024

Applying this patch make background_fill test pass in my ubuntu22 environment

@shinrich
Copy link
Member

shinrich commented Feb 9, 2024

I'm curious why the stderr error output changes so dramatically. E.g., why a purge return in one but not the other.

@duke8253 duke8253 modified the milestones: 10.0.0, 10.1.0 Apr 29, 2024
@duke8253 duke8253 merged commit 4d0f7c9 into apache:master Apr 29, 2024
@duke8253 duke8253 mentioned this pull request Apr 29, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@shinrich shinrich shinrich approved these changes

@moonchen moonchen Awaiting requested review from moonchen

Assignees

@duke8253 duke8253

Labels

AuTest Ubuntu Bugs related to Ubuntu Linux

Projects

None yet

Milestone

10.1.0

Development

Successfully merging this pull request may close these issues.

Background_fill autest fails on ubuntu22

3 participants

@duke8253 @moonchen @shinrich